Route pdf-summary discussions to Audits category#44092
Conversation
|
Hey This PR was just opened and currently has 0 changed files, so a few things to address once the implementation lands:
If you'd like a hand finishing this up, you can assign this prompt to your coding agent:
|
Add `category: "audits"` to pdf-summary.md's create-discussion config. This was the only workflow with create-discussion lacking an explicit category, causing its discussions to fall back to "Announcements" via the resolveCategoryId fallback. All other audit/analysis report workflows already route to "Audits" via the shared daily-audit-discussion.md template or explicit config. Recompile pdf-summary.lock.yml to reflect the change. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
🤖 Triage Assessment
Summary: Routes pdf-summary discussions to Audits category. Draft. Small config change (+7/−6, 2 files). No CI triggered. Next: Convert to ready-for-review when tested.
|
There was a problem hiding this comment.
Pull request overview
This PR ensures the pdf-summary workflow’s create-discussion safe-output explicitly routes generated discussions to the Audits category, avoiding the current fallback behavior that places uncategorized discussions into Announcements.
Changes:
- Added
category: "audits"tosafe-outputs.create-discussioninpdf-summary.md. - Recompiled
pdf-summary.lock.ymlso the generated safe-outputs handler config and tool descriptions include the Audits category routing.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/pdf-summary.md |
Sets create-discussion safe-output category to "audits" so discussions land in Audits rather than the fallback category. |
.github/workflows/pdf-summary.lock.yml |
Updates the generated workflow to reflect the new create_discussion.category setting and related generated metadata/config. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Low
Agent-generated report discussions were scattered across Audits (31), Announcements (29), and General (25) categories. When no
categoryis configured,resolveCategoryIdfalls back to "Announcements" — so uncategorized workflows silently land there instead of "Audits".Changes
pdf-summary.md: Addedcategory: "audits"to thecreate-discussionsafe-outputs config — the only workflow with acreate-discussionhandler config missing an explicit category.pdf-summary.lock.yml: Recompiled to reflect the updated config.All other audit/analysis workflows (including
repository-quality,security-observability,geo-optimizer,terminal-stylist) already route to Audits via theshared/daily-audit-discussion.mdimport chain.